@bbaldino: What do you think about your PR now?

I was just going through some old emails I hadn't gotten to and saw this, sorry for dropping the ball. I'm afraid I've forgotten most of the context, but do remember we did some enhancements on when to retry vs not for Jibri. @aaronkvanmeerten or @bgrozev
do you maybe remember anything around this?

I don't remember the reasons for this. The jibri's in use have definitely been updated by now, but it's hard to verify that no code ever sets failureReason without setting shouldRetry (but I trust you did that when you initially opened the PR)

it's hard to verify that no code ever sets failureReason without setting shouldRetry (but I trust you did that when you initially opened the PR)

probably? :) but i've got the same concern. if things are working as-is, maybe it's best to leave it (i.e. not merge this)--at least not without someone doing some investigation to confirm.